Skip to content

Conversation

nbpatel
Copy link
Contributor

@nbpatel nbpatel commented Aug 13, 2025

This PR adds pattern to distribute the load/store/prefetch nd ops with offsets from workgroup to subgroup IR. This PR is part of the transition to move offsets from create_nd to load/store/prefetch nd ops.

Create_nd PR : #152351

@nbpatel nbpatel marked this pull request as ready for review August 13, 2025 22:56
Copy link
Contributor

@chencha3 chencha3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM w/ minor comments.

@Garra1980
Copy link

This PR adds pattern to distribute the load/store/prefetch nd ops with offsets from workgroup to subgroup IR. This PR is part of the transition to move offsets from create_nd to load/store/prefetch.

can you please also add a link to the corresponding PR with create_nd

@nbpatel nbpatel requested a review from Jianhui-Li August 16, 2025 02:06
SmallVector<OpFoldResult> wgOffsets = getWgOffsets(op, rewriter);

// Calculate the global offsets
auto globalOffsets =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about call it allOffsets instead of globalOffsets? Not sure what global means here.

Copy link
Contributor

@Jianhui-Li Jianhui-Li left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with some minor comments.

@nbpatel nbpatel merged commit 4a9d038 into llvm:main Aug 18, 2025
9 checks passed
@nbpatel nbpatel deleted the xegpu-load-nd-offset branch September 25, 2025 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants